Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

libbpf-tools: add CO-RE 'pidpersec' #4963

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

myhro
Copy link
Contributor

@myhro myhro commented Apr 12, 2024

This is a CO-RE port of the Brendan Gregg's pidpersec BCC Python tool.

It's my second attempt in porting a BCC tool to CO-RE. I'm starting from the smaller ones, but the intention is to eventually get to the larger ones.

This is a CO-RE port of the Brendan Gregg's pidpersec BCC Python tool.
@yonghong-song
Copy link
Collaborator

Thanks for contribution. But bcc tools are used for production debugging and observations. You need to provide real use cases for the tool to be merged.

@myhro
Copy link
Contributor Author

myhro commented Apr 15, 2024

Thanks for the feedback, @yonghong-song. This PR is not a completely new addition, but a C/libbpf port of the existing pidpersec.py tool, which is documented in the companion example.

My understanding is that if a tool is already part of the repository, it's considered useful for production debugging/observations. Maybe this case is different, considering it was added ~9 years ago, and it was just a showcase of BPF functionality back then?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants